Skip to content

update icon: c (original) #2542

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
Jorenar wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

update icon: c (original) #2542

Jorenar wants to merge 1 commit into from
+7 −6

Conversation

@Jorenar
Copy link
Contributor

@Jorenar Jorenar commented Aug 27, 2025 •
edited
Loading

  • PR does not match another non-stale PR currently opened

Notes

At the first glance, in small scale, it doesn't look so bad, but when looking closer on c-line.svg, it's noticeable it got quite warped after running it through SVGOMG. Taking other icons as example, even more complicated ones, seems like the original C logo is already simple enough to fill the role of "line" icon.

Changes

  • Remove c-line.svg

@Jorenar Jorenar force-pushed the hotfix/c-rm-line branch 2 times, most recently from f0f8712 to 241e74a Compare November 2, 2025 17:37
@github-actions
Copy link
Contributor

github-actions bot commented Nov 2, 2025

Hi!

I'm the check-bot and we have some issues with your PR:

devicon.json is not sorted correctly.
Please make sure that your icon is added in the `devicon.json` file at the correct alphabetic position
as seen here: https://github.com/devicons/devicon/wiki/Updating-%60devicon.json%60

Check our CONTRIBUTING guide for more details regarding these errors.

Please address these issues. When you update this PR, I will check your SVGs again.

Thanks for your help,
SVG-Checker Bot 😄

1 similar comment
@github-actions
Copy link
Contributor

github-actions bot commented Nov 18, 2025

Hi!

I'm the check-bot and we have some issues with your PR:

devicon.json is not sorted correctly.
Please make sure that your icon is added in the `devicon.json` file at the correct alphabetic position
as seen here: https://github.com/devicons/devicon/wiki/Updating-%60devicon.json%60

Check our CONTRIBUTING guide for more details regarding these errors.

Please address these issues. When you update this PR, I will check your SVGs again.

Thanks for your help,
SVG-Checker Bot 😄

@Jorenar Jorenar force-pushed the hotfix/c-rm-line branch 2 times, most recently from 241e74a to 2e56c53 Compare November 18, 2025 03:54
@github-actions
Copy link
Contributor

github-actions bot commented Nov 18, 2025

Hi!

I'm the check-bot and we have some issues with your PR:

devicon.json is not sorted correctly.
Please make sure that your icon is added in the `devicon.json` file at the correct alphabetic position
as seen here: https://github.com/devicons/devicon/wiki/Updating-%60devicon.json%60

Check our CONTRIBUTING guide for more details regarding these errors.

Please address these issues. When you update this PR, I will check your SVGs again.

Thanks for your help,
SVG-Checker Bot 😄

1 similar comment
@github-actions
Copy link
Contributor

github-actions bot commented Nov 18, 2025

Hi!

I'm the check-bot and we have some issues with your PR:

devicon.json is not sorted correctly.
Please make sure that your icon is added in the `devicon.json` file at the correct alphabetic position
as seen here: https://github.com/devicons/devicon/wiki/Updating-%60devicon.json%60

Check our CONTRIBUTING guide for more details regarding these errors.

Please address these issues. When you update this PR, I will check your SVGs again.

Thanks for your help,
SVG-Checker Bot 😄

@Jorenar
update icon: c (original)
80eefac 
Remove unnecessary (and mangled) line version of C logo
@canaleal canaleal added the feature:icon PR when a new icon is ready to be added to the collection label Dec 22, 2025
@Jorenar
Copy link
Contributor Author

Jorenar commented Jan 11, 2026

@Snailedlt @canaleal @lunatic-fox @weh
Good to merge?

@weh
Copy link
Contributor

weh commented Jan 12, 2026

Hi @Jorenar, i prefer fixing things, instead of deleting. This version of the line icon should be better. Please check it and add it to your merge request.

<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 128 128"><path d="M62.95 0C45.98 0 30.933 6.004 20.154 17.035 9.374 28.065 2.888 44.095 2.888 64c0 19.402 5.987 35.438 17.024 46.6C30.948 121.762 46.99 128 66.888 128c31.898 0 55.092-19.156 58.089-50.121l.104-1.079H85.76l-.148.808c-.963 5.291-2.884 10.043-5.918 13.427-3.033 3.383-7.144 5.457-12.806 5.457-10.088 0-15.535-3.702-18.707-9.46-3.173-5.757-3.939-13.736-3.939-22.047 0-11.21 1.969-19.65 5.508-25.195 3.54-5.544 8.558-8.282 15.17-8.282 6.223 0 10.857 1.651 14.22 4.782 3.363 3.131 5.505 7.82 6.468 14.075l.129.835h39.375l-.139-1.108c-1.991-15.931-8.005-28.514-18.317-37.065C96.344 4.475 81.813 0 62.95 0Zm0 1.97c18.552 0 32.576 4.385 42.448 12.572 9.624 7.981 15.315 19.678 17.402 34.689H87.375c-1.072-6.131-3.286-11.026-6.892-14.383-3.776-3.515-8.987-5.31-15.564-5.31-7.173 0-12.985 3.17-16.83 9.195-3.846 6.024-5.816 14.814-5.816 26.252 0 8.427.711 16.694 4.185 22.998 3.473 6.304 9.842 10.479 20.43 10.479 6.154 0 10.905-2.357 14.272-6.112 3.193-3.562 5.099-8.371 6.153-13.58h35.504c-3.35 29.407-25.202 47.26-55.929 47.26-19.487 0-34.952-6.07-45.576-16.815C10.687 98.47 4.858 82.998 4.858 64c0-19.479 6.314-34.957 16.704-45.588C31.952 7.78 46.443 1.969 62.95 1.969Z" style="fill:#a9bacd;stroke-width:.984615"/></svg>

@Jorenar
Copy link
Contributor Author

Jorenar commented Jan 12, 2026

@weh Honestly, I'd rather see only one version of the logo floating around, but if you insist then I can update the PR later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

feature:icon PR when a new icon is ready to be added to the collection

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Jorenar @weh @canaleal